Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Jan 17, 2026

Add the no-param-reassign rule to oxlint configuration. This prevents reassignment of function parameters, enforcing immutability and avoiding subtle bugs caused by unexpected parameter mutation.


Summary by cubic

Enable the no-param-reassign rule in oxlint to block reassigning function parameters. This enforces immutability and prevents subtle bugs from unexpected parameter mutation.

Written for commit 7fbbab6. Summary will update on new commits.

Enable the no-param-reassign rule in oxlint to prevent function
parameter reassignment. This enforces immutability of function
parameters, improving code predictability and avoiding subtle bugs.
Copilot AI review requested due to automatic review settings January 17, 2026 22:13
@ryoppippi ryoppippi requested a review from a team as a code owner January 17, 2026 22:13
@ryoppippi ryoppippi merged commit 4e00d4d into main Jan 17, 2026
19 checks passed
@ryoppippi ryoppippi deleted the chore/add-no-param-reassign-rule branch January 17, 2026 22:13
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds the no-param-reassign rule to the oxlint configuration to enforce immutability and prevent subtle bugs caused by parameter mutation.

Changes:

  • Added no-param-reassign rule with "error" severity level to the restriction rules section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants